Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

AntiMicroX migration message #416

Merged
merged 3 commits into from
Nov 6, 2022
Merged

AntiMicroX migration message #416

merged 3 commits into from
Nov 6, 2022

Conversation

pktiuk
Copy link
Contributor

@pktiuk pktiuk commented Nov 4, 2022

Linked with: #351

Adds message informing about the end of further development and informing about AntiMicroX.

@pktiuk
Copy link
Contributor Author

pktiuk commented Nov 4, 2022

@gombosg

image

What do you think about this form of informing about end of further development?
And what do you think about this phrasing?
Maybe "This app would be no longer developed, press here for more information"...

This button would open a window with a bit more elaborate description of situation and inside of that window there would be a hyperlink leading to antimicrox githup page.

@gombosg
Copy link
Contributor

gombosg commented Nov 4, 2022

Hey @pktiuk !

⚠️ Antimicro is no longer maintained and won't receive any more updates.
Click here to get AntimicroX, an updated fork of the app.

IMO

  • a separate popup is not even needed, just go straight away to the github/app page. Much easier to do.
  • We could use some emoji if it works 😛

@pktiuk pktiuk force-pushed the migration_message branch from de66e2b to ca6db45 Compare November 4, 2022 21:13
@pktiuk pktiuk force-pushed the migration_message branch from 7d162ed to aac3434 Compare November 4, 2022 21:45
@pktiuk
Copy link
Contributor Author

pktiuk commented Nov 4, 2022

We could use some emoji if it works stuck_out_tongue

Good idea

image
I think, that it is better to show a bit more elaborate description of this why they should migrate (most of these users probably don't know what a fork is) and present some benefits.

@pktiuk pktiuk marked this pull request as ready for review November 4, 2022 21:49
@pktiuk pktiuk changed the title first draft of migration message AntiMicroX migration message Nov 4, 2022
Copy link
Contributor

@gombosg gombosg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@pktiuk pktiuk merged commit 1f0dfad into master Nov 6, 2022
@pktiuk pktiuk deleted the migration_message branch November 7, 2022 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants